Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use Integer instead of int in KafkaClusterSpec replicas #9663

Merged
merged 3 commits into from
Feb 11, 2024

Conversation

im-konge
Copy link
Member

@im-konge im-konge commented Feb 9, 2024

Type of change

  • Enhancement

Description

This PR changes the int as a type for replicas to Integer, so we can set it as null when the KafkaBuilder for the Kafka object is used -> for applying using fabric8 API.
It also contains few null checks.

Checklist

  • Make sure all tests pass

Signed-off-by: Lukas Kral <[email protected]>
@im-konge im-konge added this to the 0.40.0 milestone Feb 9, 2024
@im-konge im-konge marked this pull request as ready for review February 9, 2024 10:43
@im-konge im-konge self-assigned this Feb 9, 2024
Signed-off-by: Lukas Kral <[email protected]>
@im-konge
Copy link
Member Author

im-konge commented Feb 9, 2024

/azp run regression

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@scholzj
Copy link
Member

scholzj commented Feb 10, 2024

/azp run kraft-regression

@im-konge
Copy link
Member Author

/azp run kraft-regression

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@scholzj scholzj merged commit ac92161 into strimzi:main Feb 11, 2024
29 checks passed
@im-konge im-konge deleted the kafka-integer branch February 11, 2024 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants